Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Provisioning test workflow names #14837

Merged

Conversation

shubhamsg199
Copy link
Contributor

Problem Statement

The workflow names were recently updated and now they have iac prepended to the them, the tests were failing because we had old workflow names in the test.

Solution

Update them to the correct workflow names

@shubhamsg199 shubhamsg199 added CherryPick PR needs CherryPick to previous branches 6.12.z Introduced in or relating directly to Satellite 6.12 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 labels Apr 19, 2024
@shubhamsg199 shubhamsg199 self-assigned this Apr 19, 2024
@shubhamsg199 shubhamsg199 requested a review from a team as a code owner April 19, 2024 07:10
Copy link
Contributor

@shweta83 shweta83 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack

@ogajduse
Copy link
Member

Can you consider making the WF name configurable through settings?

@shubhamsg199 shubhamsg199 force-pushed the provisioning-iac-workflow-fix branch from 7ee5ab7 to d05432a Compare April 22, 2024 10:33
@shubhamsg199
Copy link
Contributor Author

Can you consider making the WF name configurable through settings?

Updated

@shubhamsg199 shubhamsg199 force-pushed the provisioning-iac-workflow-fix branch from d05432a to a8117f1 Compare April 22, 2024 11:11
@Gauravtalreja1 Gauravtalreja1 added Easy Fix :) Easiest Fix to review and quick merge request. and removed AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing labels Apr 22, 2024
@Gauravtalreja1 Gauravtalreja1 enabled auto-merge (squash) April 22, 2024 11:14
@Gauravtalreja1 Gauravtalreja1 merged commit 546bc75 into SatelliteQE:master Apr 22, 2024
10 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 22, 2024
Provisioning test workflow name changes

Signed-off-by: Shubham Ganar <[email protected]>
(cherry picked from commit 546bc75)
github-actions bot pushed a commit that referenced this pull request Apr 22, 2024
Provisioning test workflow name changes

Signed-off-by: Shubham Ganar <[email protected]>
(cherry picked from commit 546bc75)
Gauravtalreja1 pushed a commit that referenced this pull request Apr 22, 2024
Update Provisioning test workflow names (#14837)

Provisioning test workflow name changes

Signed-off-by: Shubham Ganar <[email protected]>
(cherry picked from commit 546bc75)

Co-authored-by: Shubham Ganar <[email protected]>
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
Provisioning test workflow name changes

Signed-off-by: Shubham Ganar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.12.z Introduced in or relating directly to Satellite 6.12 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 CherryPick PR needs CherryPick to previous branches Easy Fix :) Easiest Fix to review and quick merge request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants